Skip to content

docs: improve S2 docs markdown / agent skill#9939

Merged
reidbarber merged 23 commits into
mainfrom
s2-skill-fixes
May 15, 2026
Merged

docs: improve S2 docs markdown / agent skill#9939
reidbarber merged 23 commits into
mainfrom
s2-skill-fixes

Conversation

@reidbarber
Copy link
Copy Markdown
Member

@reidbarber reidbarber commented Apr 16, 2026

Includes improvements to the markdown docs output and S2 agent skill. Also adds a comment to PRs to show the diff in the agent skills.

Markdown output improvements:

  • skip redirect pages (the mcp.mdx pages got replaced by the ai.mdx pages)
  • skip error pages (no need for errror.md files)
  • fix inline code being stripped from page descriptions (visible here)
  • include Routers section in S2 getting-started.md (renders at bottom of the page once and gets referenced above)
  • improve style-macro.md (more clear and concise, uses aliases for repetitive sets)

Skill Improvements:

  • fix heading levels in component-decision-tree.md
  • improve illustration example in implementation-guidance.md to include all three types of illustrations
  • improve skill with guidance for using Buttons with icons + text (ensure text is wrapped in Text)
  • improve skill with guidance to avoid concatenating class names from the style macro
  • add a figma-to-s2.md resource to help with implementing designs with the Figma MCP
  • add a creating-custom-components.md resource for showing RAC + style macro usage

General docs improvements:

  • add 'dnd' tag for RAC dnd docs page (so it shows up if you search 'dnd' in the docs)
  • mention passing isFocusVisible in JSDoc for focusRing

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

View markdown docs output and skill output for affected files.

To see the agent skill diff, first make sure you have the latest release's skill installed:

npx skills add https://react-spectrum.adobe.com --skill react-spectrum-s2

then install this PR's and look at the diff:

npx skills add https://d1pzu54gtk2aed.cloudfront.net/pr/91dfb8d78bbe5b9a0a3b1bd48f8ff9fc784c023c/ --skill react-spectrum-s2

🧢 Your Project:

@github-actions github-actions Bot added the S2 label Apr 16, 2026
* add skill differ

* improve comment

* deploy full diff + link to new/changed files

* fix path

* remove full diff output

* comment with install links
@rspbot
Copy link
Copy Markdown

rspbot commented Apr 20, 2026

@rspbot
Copy link
Copy Markdown

rspbot commented May 11, 2026

devongovett
devongovett previously approved these changes May 11, 2026
# Conflicts:
#	packages/dev/s2-docs/scripts/generateMarkdownDocs.mjs
@rspbot
Copy link
Copy Markdown

rspbot commented May 14, 2026

@rspbot
Copy link
Copy Markdown

rspbot commented May 14, 2026

@rspbot
Copy link
Copy Markdown

rspbot commented May 14, 2026

@rspbot
Copy link
Copy Markdown

rspbot commented May 14, 2026

@rspbot
Copy link
Copy Markdown

rspbot commented May 14, 2026

Agent Skills Changes

Added (2)
Removed (2)
  • react-aria/skills/react-aria/references/components/mcp.md
  • s2/skills/react-spectrum-s2/references/components/mcp.md
Modified (70)
Install

React Spectrum S2:

npx skills add https://d1pzu54gtk2aed.cloudfront.net/pr/91dfb8d78bbe5b9a0a3b1bd48f8ff9fc784c023c/

React Aria:

npx skills add https://d5iwopk28bdhl.cloudfront.net/pr/91dfb8d78bbe5b9a0a3b1bd48f8ff9fc784c023c/

Copy link
Copy Markdown
Member

@LFDanLu LFDanLu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, did we want to add the skeleton collection reference anywhere here or maybe not applicable until it is more widely supported?

@reidbarber reidbarber added this pull request to the merge queue May 15, 2026
@reidbarber
Copy link
Copy Markdown
Member Author

@LFDanLu Yeah felt weird to mention since it is just for CardView now. I'll make a note to include it in a future update.

Merged via the queue into main with commit 6d4afb9 May 15, 2026
33 checks passed
@reidbarber reidbarber deleted the s2-skill-fixes branch May 15, 2026 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants